Skip to content

Add build test for PRs#3011

Merged
evilnick merged 2 commits inton8n-io:mainfrom
evilnick:buildcheck
Mar 24, 2025
Merged

Add build test for PRs#3011
evilnick merged 2 commits inton8n-io:mainfrom
evilnick:buildcheck

Conversation

@evilnick
Copy link
Contributor

This adds a workflow to check that the site builds in strict mode, which should normally be the case. The main purpose of this is to catch internal link fails on PRs.

  • adds a workflow
  • mandates 1.61 version of mkdocs
  • It needs it's own mkdocs.yml file (mkdocs-test.yml) so that:
    • it can build without the material-preview
    • it can save time by cutting out some other things which increase build time but won't result in link errors etc.
  • to facilitate above and to prevent drift, this pr also separates the nav from the main mkdocs.yml

the workflow will remain silent unless the strict build fails, in which case it dumps the mkdocs output into a comment and marks the workflow as failed

@evilnick evilnick requested a review from imchairmanm March 23, 2025 18:14
@netlify
Copy link

netlify bot commented Mar 23, 2025

Deploy Preview for n8n-docs ready!

Name Link
🔨 Latest commit 9ca2f73
🔍 Latest deploy log https://app.netlify.com/sites/n8n-docs/deploys/67e148cbd90e1300083327a3
😎 Deploy Preview https://deploy-preview-3011--n8n-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@imchairmanm imchairmanm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks interesting. I'm curious to see it in action!

@evilnick
Copy link
Contributor Author

@evilnick evilnick merged commit 14c61f0 into n8n-io:main Mar 24, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants